Skip to content

Make onSelect works also for object keys#101

Merged
Kikobeats merged 10 commits intomicrolinkhq:masterfrom
andres-artunduaga:issue-82
Mar 5, 2026
Merged

Make onSelect works also for object keys#101
Kikobeats merged 10 commits intomicrolinkhq:masterfrom
andres-artunduaga:issue-82

Conversation

@andres-artunduaga
Copy link
Contributor

Thanks for reviewing this.

Sending code to cover #82

1. Key Click Selection Behavior

  • Added key-click selection support so clicking object or array keys triggers onSelect, not only value clicks.
  • Applied this behavior consistently across ArrayGroup, JsonObject, ObjectName, and VariableEditor.
  • Added unit tests to cover key-click selection paths.

2. Lint and Test Cleanup

  • Applied style and lint compliance fixes required by pre-commit checks.
  • Cleaned naming, equality checks, handler references, and unused code paths.
  • Updated related tests to keep formatting and fixtures consistent with lint rules.

3. Unit Test Assertion Follow-up

  • Fixed one JsonObject unit test assertion for expanded icon count after test fixture normalization.
  • Confirmed unit tests pass after this adjustment.

@vercel
Copy link

vercel bot commented Feb 27, 2026

@andres-artunduaga is attempting to deploy a commit to the Microlink Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Feb 28, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
react-json-view Ready Ready Preview, Comment Mar 5, 2026 10:16am

Request Review

@Kikobeats
Copy link
Member

@andres-artunduaga thanks, can you add it as part of the demo? 🙏

@Kikobeats
Copy link
Member

@cursor review

@Kikobeats
Copy link
Member

@andres-artunduaga argg something is broken, can you take a look? 🙏

@andres-artunduaga
Copy link
Contributor Author

@Kikobeats I've noticed some linting errors during the push process. I've sent a few commits to resolve these. This should keep the code without lint issues for future contributors.

@Kikobeats Kikobeats merged commit 7c81ae7 into microlinkhq:master Mar 5, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants